<html>
<head><meta charset="utf-8"><title>Clippy Lints · t-compiler/rust-analyzer · Zulip Chat Archive</title></head>
<h2>Stream: <a href="https://rust-lang.github.io/zulip_archive/stream/185405-t-compiler/rust-analyzer/index.html">t-compiler/rust-analyzer</a></h2>
<h3>Topic: <a href="https://rust-lang.github.io/zulip_archive/stream/185405-t-compiler/rust-analyzer/topic/Clippy.20Lints.html">Clippy Lints</a></h3>

<hr>

<base href="https://rust-lang.zulipchat.com">

<head><link href="https://rust-lang.github.io/zulip_archive/style.css" rel="stylesheet"></head>

<a name="211987620"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/185405-t-compiler/rust-analyzer/topic/Clippy%20Lints/near/211987620" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Wesley Norris <a href="https://rust-lang.github.io/zulip_archive/stream/185405-t-compiler/rust-analyzer/topic/Clippy.20Lints.html#211987620">(Oct 01 2020 at 23:14)</a>:</h4>
<p>Is there any desire to go through and fix most of the clippy lints (at least ones that don't require refactoring other parts of the codebase or change behavior)? noticed there's quite a few when I opened up the project today, so I wouldn't mind going through and tidying it up :D (the vast majority are auto-fixable it seems)</p>



<a name="211998390"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/185405-t-compiler/rust-analyzer/topic/Clippy%20Lints/near/211998390" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Jeremy Kolb <a href="https://rust-lang.github.io/zulip_archive/stream/185405-t-compiler/rust-analyzer/topic/Clippy.20Lints.html#211998390">(Oct 02 2020 at 02:30)</a>:</h4>
<p>I have a PR up for some performance related ones (<a href="https://github.com/rust-analyzer/rust-analyzer/issues/6104">rust-analyzer#6104</a>)</p>



<a name="211998524"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/185405-t-compiler/rust-analyzer/topic/Clippy%20Lints/near/211998524" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Jeremy Kolb <a href="https://rust-lang.github.io/zulip_archive/stream/185405-t-compiler/rust-analyzer/topic/Clippy.20Lints.html#211998524">(Oct 02 2020 at 02:32)</a>:</h4>
<p>As for others go for it. @matklad is fairly opinionated on clippy lints so will give you good feedback</p>



<a name="212044944"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/185405-t-compiler/rust-analyzer/topic/Clippy%20Lints/near/212044944" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> matklad <a href="https://rust-lang.github.io/zulip_archive/stream/185405-t-compiler/rust-analyzer/topic/Clippy.20Lints.html#212044944">(Oct 02 2020 at 08:14)</a>:</h4>
<p><span class="user-mention" data-user-id="228507">@Wesley Norris</span> I've documented our Clippy policy at <a href="https://github.com/rust-analyzer/rust-analyzer/pull/6118">https://github.com/rust-analyzer/rust-analyzer/pull/6118</a></p>



<hr><p>Last updated: Aug 07 2021 at 22:04 UTC</p>
</html>